Skip to content

Add unified navigation test playbook and umbrella components#173

Open
JakeSCahill wants to merge 29 commits into
mainfrom
feature/unified-navigation-playbooks
Open

Add unified navigation test playbook and umbrella components#173
JakeSCahill wants to merge 29 commits into
mainfrom
feature/unified-navigation-playbooks

Conversation

@JakeSCahill
Copy link
Copy Markdown
Contributor

@JakeSCahill JakeSCahill commented May 12, 2026

Adds a single consolidated test playbook for unified navigation testing with all remote feature branches.

Changes

New Test Playbook: test-unified-nav.yml

Replaces 4 separate test playbooks with one comprehensive testing configuration:

Content Sources (Remote Feature Branches):

  • docs: feature/page-header-data (main) + 7 version branches
    • feature/rename-streaming-23.3
    • feature/rename-streaming-24.1
    • feature/rename-streaming-24.2
    • feature/rename-streaming-24.3
    • feature/rename-streaming-25.1
    • feature/rename-streaming-25.2
    • feature/rename-streaming-25.3
  • cloud-docs: feature/component-home-v3
  • rp-connect-docs: feature/unified-navigation
  • redpanda-labs: HEAD (local)
  • adp-docs: HEAD (local)

Extensions:

  • Unified navigation extension (local path for testing)
  • All essential extensions for full functionality

Start Page: data-platform::index.adoc (Data Platform landing page)

New Umbrella Components

data-platform/

Data Platform umbrella component:

  • Overview page with navigation to Streaming and Connect
  • Minimal nav structure
  • Enables unified sidebar aggregation

self-managed/

Self-Managed parent component:

  • Overview page for hierarchical organization
  • Minimal nav structure
  • Parent for Streaming and Connect in hierarchy

Related PRs

Extension & UI

Content Repositories

Testing Coverage

This playbook enables testing of:

  • ✅ Unified sidebar with multiple buckets (Streaming, Connect)
  • ✅ Component theming (colors, icons)
  • ✅ Per-bucket version selectors
  • ✅ Bucket expand/collapse
  • ✅ Component home v3 layouts
  • ✅ Data Platform landing page
  • ✅ All 7 renamed version branches
  • ✅ Product switcher with umbrella hierarchy
  • ✅ Navigation aggregation from child components

- Add test-unified-nav.yml: Single test playbook for unified navigation
  - Uses remote feature branches from all repos
  - Includes all 7 version branches: v23.3, v24.1-24.3, v25.1-25.3
  - References unified-navigation extension
  - Starts at data-platform landing page

- Add data-platform umbrella component
  - Overview page with links to child components
  - Minimal navigation structure

- Add self-managed parent component
  - Overview page for Self-Managed hierarchy
  - Minimal navigation structure

- Remove old test playbooks (test-cloud-only, test-labs, test-multicomponent, test-simple)

This playbook enables testing of the complete unified navigation implementation
with all remote feature branches.
@JakeSCahill JakeSCahill requested a review from a team as a code owner May 12, 2026 10:38
@netlify
Copy link
Copy Markdown

netlify Bot commented May 12, 2026

Deploy Preview for redpanda-documentation ready!

Name Link
🔨 Latest commit 9259036
🔍 Latest deploy log https://app.netlify.com/projects/redpanda-documentation/deploys/6a0c20ac146d665c395225e5
😎 Deploy Preview https://deploy-preview-173--redpanda-documentation.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Change from local path to git+https reference so the playbook can be
used by anyone without needing the local extension repo.

References feature/unified-navigation branch directly from GitHub.
- Update @redpanda-data/docs-extensions-and-macros to git reference
- Points to feature/unified-navigation branch for testing
- Lock dependencies with package-lock.json
The glob stack overflow was caused by using local filesystem paths
which exposed node_modules with circular symlinks (109 in adp-docs,
many in labs Chrome Framework).

Using remote GitHub URLs fixes this because git repos don't include
node_modules, eliminating the circular reference issue.

Changes:
- redpanda-labs: local path → github.com/redpanda-data/redpanda-labs
- adp-docs: local path → github.com/redpanda-data/adp-docs

Unified navigation extension now loads successfully.
- Upgraded @antora/cli and @antora/site-generator from 3.1.2 to 3.1.14
- Changed test playbook to use local paths for all repos
- Added exclude patterns for node_modules (though upgrade made them unnecessary)
- This allows testing local repos without glob stack overflow from circular symlinks
- Build now progresses to show actual include errors that need fixing
@JakeSCahill
Copy link
Copy Markdown
Contributor Author

JakeSCahill commented May 12, 2026

Jira: DOC-2180

JakeSCahill and others added 22 commits May 12, 2026 12:41
- Using local paths for docs, cloud-docs, rp-connect-docs, labs, adp-docs
- This allows testing xref fixes locally before pushing
- Antora 3.1.14 handles local repos without glob overflow
- Build completes successfully with unified navigation
Use sentence case for section headings:
- "Choose Your Component" → "Choose your component"
- "Deployment Options" → "Deployment options"

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Change colons to forward slashes for subdirectory paths
- Fix BYOC link: cluster-types/byoc/index.adoc (was cluster-types:byoc:index.adoc)
- Fix Connect quickstart: get-started/quickstarts/index.adoc
- Fix producer/consumer config paths with forward slashes
- Fix schema registry path: manage/schema-reg/index.adoc

All links now resolve correctly from versionless data-platform component.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- More accurate name: this is component-level metadata, not page-level
- Update all three umbrella components: home, data-platform, self-managed
- Remove section field (deprecated - breadcrumbs now use page-navigation config)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Add expand-buckets attribute for ADP bucket
- Add hero section with eyebrow, title, subtitle
- Add intent cards section (Data Platform and ADP)
- Add products section with component cards
- Swap intent cards: ADP first, then Data Platform
- Swap product cards: ADP first in hierarchy

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Changes to all three playbooks (preview, local, antora):

UI Bundle:
- Use v3.0.0-beta1 beta release (includes breaking changes)
- Cloud/Self-Managed card swap on Data Platform page
- component-metadata rename (was page-header-data)
- Config-driven unified navigation

Content Sources:
- Add data-platform and self-managed umbrella components
- Use feature/rename-streaming-main branch for current docs
- Use feature/rename-streaming-* branches for versioned docs (23.3-25.3)
- Use feature/component-home-v3 for cloud-docs
- Use feature/unified-navigation for rp-connect-docs
- Use feature/platform-badges for adp-docs

Extensions:
- Add unified-navigation extension (config-driven architecture)
- Extensions loaded from local file path (feature/unified-navigation branch)

Testing:
- All WIP branches contain component-metadata (not page-header-data)
- All umbrella components have page-navigation config
- Breadcrumb hierarchy computed from navigation tree

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Change from local file path to GitHub branch reference:
- Was: file:../docs-extensions-and-macros (only works locally)
- Now: github:redpanda-data/docs-extensions-and-macros#feature/unified-navigation

This allows Netlify to fetch the extensions package from GitHub
during preview builds, using the WIP feature/unified-navigation branch.

Also update test playbook:
- Fix start_page: home::index.adoc (was data-platform)
- Add shared branch for global attributes

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Fix mobile nav collapse/reopen bug
- Improve light mode expand button visibility
- Make topbar fully opaque for better readability
- Reduce gap in mobile nav

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Regenerate lock file to fix afdocs CI check.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Point to feature/page-header-data branch in redpanda-labs which
has the component renamed from 'redpanda-labs' to 'labs'.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Required for search to work with renamed components.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Upgrade Node from 20 to 22 (required by dependencies)
- Add NODE_OPTIONS with 6GB heap size for Algolia indexer

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
JakeSCahill and others added 2 commits May 19, 2026 08:46
The proxy-api-docs edge function was crashing with 'uncaught exception
during edge function invocation' when accessing /api/doc/cloud-dataplane/.

Root cause: The DOMParser WASM module (deno_dom) can fail to initialize
in Netlify's edge function runtime, throwing an unhandled exception.

Solution: Wrap DOMParser instantiation in try-catch block. If parsing
fails, return unmodified HTML from Bump.sh rather than crashing the
entire edge function.

This ensures graceful degradation - API docs will still render even if
widget injection fails, rather than showing a crash page to users.

Fixes: https://deploy-preview-173--redpanda-documentation.netlify.app/api/doc/cloud-dataplane/

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant